-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nix: 2.18 -> 2.22 #315262
nix: 2.18 -> 2.22 #315262
Conversation
Removed request from Raito and Ma27, because they now switched to lix. |
one issue i noticed is when using |
This is fixed in master: NixOS/nix#10617 |
Thanks! looks like NixOS/nix#10617 was reverted in favor of NixOS/nix#10754 |
I tested this version on my system without any issues, but on the other hand it has better error handling i.e. for infinite recursion. This version has been now also cooking in the nix-install-action for 2 weeks: cachix/install-nix-action#206
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested nix repl
works great! thanks!
This regressed the |
Which blocks the unstable channels and should be fixed asap. |
Mistakes are fine but I told about the misc tests so many times about the Nix maintenance and I am quite disconcerted by this happening so obviously again. If Nix maintenance cannot have a reasonable enough QA, it's OK, there's no shame to that. In that case, I advise to switch the default implementation of Nix in Nixpkgs to be Lix, we are old Nix maintainers in Nixpkgs and we have a bit more experience on how to run this. We are back to pre-code ownership QA levels now, this is not good :/. |
Newer Nix also broke |
Critics taken, this was my first time bumping nix and it wasn't clear to me that
I would like to ask you not to cause further rift in the community. Calling Nix developer incompetent is helping no one.
|
Thanks. I wasn't aware of NixOS/nix#9652. I look into it. |
No problem, but please coordinate with the Nix maintenance team, no offense to you, Mic, but I am tired of the disorganized situation where I repeat things over months multiple times to different people. In general, upgrading the stable version of Nix is a very involved operation and maybe people should first work on writing a proper checklist before upgrading it, I am sad this didn't happen even though we discussed it in the Nix maintenance team meeting to the best of my knowledge.
This N-th untested merge is causing further rift, unfortunately. It's hardly respectful to people's time after a327fd0 and so many discussions in those sorts of PRs and the powerless feeling that no matter what you say, it seems like it is absolutely not heard or considered in any meaningful way. I am not calling Nix developers incompetent, Nix developers mentioned multiple times they did not have the bandwidth to take care of the QA process for Nix in Nixpkgs, I am just saying it is hard to hear: "We cannot take care of it" and "We will just yolo merge things" at the same time, I am offering an honest alternative option: just do not advertise something you cannot take care of. So please do not put words in my mouth about things I didn't say. What I am asking for, which I think is reasonable, for the past months, is to stop breaking people's package manager when it could have been avoided. If that is not possible, this will cause further rift, yes. |
This also broke |
Who is the Nix maintenance team? I only see ma27 and you as codeowner. |
https://nixos.org/community/teams/nix/ AFAIK, we were just Nix maintainers in Nixpkgs to help with the current state of things, but I hope that the Nix maintenance team gets more involved to provide a polished experience to everyone involved. |
You must have this checklist than. Could you give it to me? Otherwise I need to approximate to my best abilities. |
|
@Ma27 are you still maintaining nix in nixpkgs or not? |
Yes, of course, Nix maintenance in Nixpkgs starts where Nix maintenance testing stops, i.e.
I do not care about Flakes personally, so I don't do any special regression testing on this, but I would imagine that some people would do so. I do some special tests based on my own infrastructure and my own model of the current type of regressions that Nix are creating:
And that's what I can come up with, on the spot, as I am at GPN right now. A generic checklist can start like this, each Nix stable bumps are special because reading the release notes are critical to have ideas of "what to test". Reviewing the issue list with the tag bug is also important to see what are release blockers bugs or not, IMHO. With that in mind, the generic checklist can be refined to a specialized checklist to a specific bump. |
No, this is NixOS, not LixOS. You are free to create your own distro, of course. I think it would be good for the Nix team to take ownership of maintaining the Nix package in Nixpkgs, but that should be discussed in the team. It might not be realistic to make the maintainers of the Nix package responsible for every breakage in a dependent package. E.g. in the case of
I believe this is covered by the NixOS installer tests. |
I don't think it's fair to dogpile on @Mic92 when a bunch of people who were doing this are stepping down. It's reasonable, if not ideal, that human knowledge gets lost and rediscovered in the process. It may even be that bumping Nix eagerly is a good thing --- so more people are mad at the Nix team for whatever breakage there is, provided that it doesn't grind Nixpkgs itself to a halt too much. Some bugs do the latter for sure, but other bugs just do the former. |
Just for the sake of completeness, for that we do have |
My suggestion is to keep a list of consumers to ping, when upgrading the nix package manager. I for instance often use these notifications to know when I have to re-check my own packages that have tight coupling with nix as a build dependency i.e. nix-eval-jobs and harmonia. |
We could also set up a small Hydra jobset for testing Nix upgrades, similar to |
Having Lix as a default implementation does not change the fact this is NixOS, it's just not using NixOS/nix as an implementation. I do not see the problem, this is still Nix. It would be nice to provide clear arguments on why this would be a problem.
I think this was submitted multiple times to the team and I thought that @fricklerhandwerk was considering taking the ownership. I am not sure what is the blocker here.
There's plenty of ways to do it and you just mentioned one. Ideally, we review the reverse dependency closure and try to be reasonable to mention breakages to dependants. In this case, this is a release blocker, so this is a non-discussion. A release blocker needs to be tested before the PR is merged?
It tests quite basic feature, i.e. building a non-nixpkgs crafted derivation. There's room for a bit more coverage, with a UX orientation, i.e. "can I recover my NixOS system using that new Nix?" which is one clear regression we had in the past and took months to fix and we shipped a release of NixOS without this feature.
I don't really understand this argument, I thought it was pretty clear this is not a blame game, but we owe to our users a working Nix implementation. Mic92 didn't merge this, it's lovesegfault who did. Mic92 is reacting and I'm providing my input. If you are telling me this is dogpiling, then I fear that there's no room to talk about the uncomfortable truth that we are running at the risk of breaking our users all the time and this is not really serious of us. We are not that reckless with, arguably, equally important part of the distribution. Why should we be here?
I think it's important to get a coherent story straight, what you are saying feels disaligned with NixOS/nix#10288. I do not understand why this seems to be a reoccurring pattern (disalignement in the public opinions of Nix maintenance team members). It's very frustrating that I have to appear as the angry dude of this PR. I will repeat my motivations to make them clear: I am not deriving any pleasure to dunk on the Nix maintenance team or the OP of this PR or the merger of this PR. People do mistakes, that's totally fine. But I am disappointed to see yet again a communication problem just after we "step down" (the PR is not even merged, this PR was opened 2 days ago and merged quite quickly…). It seems like to me that people didn't understand why Nix stable was Nix 2.18 for a while and stayed this way and we had no plan to upgrade. What I don't understand is why people don't inquire, don't ask, etc. If that's how we are to behave in a social project on such a critical piece of software for which we don't have good recovery stories, then we must accept that we will break users and people will ragequit over this. I personally don't want have anything to do with this, that's why I am adamant about ensuring we deliver a reasonable quality. When we do systemd bumps, we try to be careful, we run more complicated smoke testing, we test more scenarios. Nix is supposed to be, to some extent, simpler than systemd. Please drop the defensive behavior, it seems needless to me, just own the responsibilities, focus on good quality releases for Nix, focus on better testing scenarios and let's have a great QA process. And immediately, I will stop feeling like having the same discussions for months and that nothing has changed and people doesn't care. Nix has no reason to be so different from the ambiant quality of critical packages in Nixpkgs. |
But that's the precise time I'd expect to see a communication problem! That's why I am not disappointed/whatever, and I don't think you should be either. |
I would assume the Nix part of NixOS is referencing to the language, not the specific implementation of it, or am I wrong on that assumption? |
This might be me but I fail to understand why we plead that it is normal to repeat the same actions and expect different outcomes… Why should I not be disappointed to see the same things happen after trying so hard to tell people that this will happen if nothing change? It is even more frustrating when folks agree there's something to change and then the same things happen to me. So, can you explain me why I should not be disappointed by the current state of things? Here's a proposal to give hope: let's pledge that we will not let this re-occur and we will be all cautious together about next bumps, we will go through the checklist and show to the community that we go through it. |
I feel like we reached a point in this discussion, where everything that was to say, has been said. I will work on some documentation similar to what we have for the Rust compiler, so that we know what needs to be tested for a nix release. |
This comment was marked as duplicate.
This comment was marked as duplicate.
I think having documentation / paper trail / automated tests, versus a series of conversations that didn't have their intended effect, will be much better. |
@kanashimia I also fixed you |
I was under the impression that the new Nix releases were already undergoing more thorough testing, and that the responsibility had been transferred to the Nix maintainers, given this has been discussed for a while. As a result, when I saw a PR from a well-known maintainer I trust, with fixes for issues in the release, and not much discussion, I merged it. I had been running 2.22 since its release, and hadn't seen major issues either. I'm sorry for the disruption, and I'll refrain from participating in future Nix bumps, it's simply not worth the stress and toxicity. |
I don't think that's the right approach, we should fix the workflow so that everyone is comfortable doing the bumps. |
I tested this version on my system without any issues, but on the other hand it has better error handling i.e. for infinite recursion.
This version has been now also cooking in the nix-install-action for 2 weeks: cachix/install-nix-action#206
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.